Skip to content

gh-138489: Add build-details.json generation to PC/layout#149153

Open
zooba wants to merge 5 commits into
python:mainfrom
zooba:gh-138489
Open

gh-138489: Add build-details.json generation to PC/layout#149153
zooba wants to merge 5 commits into
python:mainfrom
zooba:gh-138489

Conversation

@zooba
Copy link
Copy Markdown
Member

@zooba zooba commented Apr 29, 2026

@zooba
Copy link
Copy Markdown
Member Author

zooba commented Apr 29, 2026

Supersedes #138490, and deliberately omits changes to the (deprecated) MSI installer, largely because there's no real value in dealing with the complexity of having multiple overlapping installs.

@read-the-docs-community
Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #32469300 | 📁 Comparing cc522fc against main (f1588d4)

  🔍 Preview build  

14 files changed · ± 14 modified

± Modified

@zooba
Copy link
Copy Markdown
Member Author

zooba commented May 8, 2026

@hugovk Any chance of a beta 2 exception for this one (adds a new file to the file system layout)? No functional change to the runtime, and the PEP isn't new, we just never got around to implementing it for Windows until now.

@hugovk
Copy link
Copy Markdown
Member

hugovk commented May 8, 2026

@zooba Yes, that's fine. Thanks for working on this.

@zooba
Copy link
Copy Markdown
Member Author

zooba commented May 11, 2026

For reviewers' sake, here's a sample file as generated by this script for a regular amd64 release from main:

{
  "schema_version": "1.0",
  "base_prefix": ".",
  "base_interpreter": "python.exe",
  "platform": "win-amd64",
  "language": {
    "version": "3.16",
    "version_info": {
      "major": 3,
      "minor": 16,
      "micro": 0,
      "releaselevel": "alpha",
      "serial": 0
    }
  },
  "implementation": {
    "name": "cpython",
    "cache_tag": "cpython-316",
    "version": {
      "major": 3,
      "minor": 16,
      "micro": 0,
      "releaselevel": "alpha",
      "serial": 0
    }
  },
  "abi": {
    "flags": [],
    "extension_suffix": ".cp316-win_amd64.pyd",
    "stable_abi_suffix": ".pyd"
  },
  "suffixes": {
    "source": [
      ".py"
    ],
    "bytecode": [
      ".pyc"
    ],
    "extensions": [
      ".cp316-win_amd64.pyd",
      ".pyd"
    ]
  },
  "libpython": {
    "dynamic": "python316.dll",
    "dynamic_stableabi": "python3.dll",
    "link_extensions": true
  },
  "c_api": {
    "headers": "Include"
  }
}

@zooba zooba requested a review from chris-eibl May 14, 2026 12:50
Copy link
Copy Markdown
Member

@chris-eibl chris-eibl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I am away from my dev box, though, and thus couldn't test locally.

Comment thread PC/layout/support/builddetails.py Outdated
Comment thread PC/layout/support/builddetails.py
@zooba zooba requested a review from vsajip as a code owner May 14, 2026 22:31
@zooba
Copy link
Copy Markdown
Member Author

zooba commented May 14, 2026

Whoops, pushed test fixes for the wrong PR. Reverted now - sorry Vinay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants